Skip to content

U of T DSI | Shell - Assignment #1 (Part 1 & 2)#1

Open
StephanieShishis wants to merge 4 commits into
mainfrom
assignment
Open

U of T DSI | Shell - Assignment #1 (Part 1 & 2)#1
StephanieShishis wants to merge 4 commits into
mainfrom
assignment

Conversation

@StephanieShishis

Copy link
Copy Markdown
Owner

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

In the script, I needed to add code that allowed me to organize raw data into folders. These folders were specific subdirectories based on server, user and event log files. I also removed files containing IP addresses for privacy. Lastly, I created an inventory file that listed a summary of all the processed files.

What did you learn from the changes you have made?

I was able to use Bash commands that I learned in the live learning session such as mkdir, mv, cp, rm, ls, touch, > and >>. I also was able to get more comfortable working both in the terminal and open coding files to create a script.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

I could have used the find command to list all files instead of listing each folder with ls. I also think there were some better ways to complete some tasks in fewer steps but any advice on this would be appreciated.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

A big challenge was remembering all the commands from the live learning lectures and also ensuring I was creating folders in the right directories. cd was a super useful function and helped me remember where I was in the code.

How were these changes tested?

I tested the script using bash assignment.sh before committing any changes and double checked my user profile on my remote desktop to ensure the folders were created correctly and it contained the right information.

A reference to a related issue in your repository (if applicable)

Checklist

  • [YES] I can confirm that my changes are working as intended

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autograder results

question status comment
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2 coworker-changes branch not found in commit history

Please address the issues listed above.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autograder results

question status
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2

@StephanieShishis

Copy link
Copy Markdown
Owner Author

For Part 2: I modified my coworkers code as they used the incorrect command rf instead of rm. I fixed the section and finished the merge by saving the file, using git add and git commit. I then re-tested the code in the terminal to make sure it worked.

@StephanieShishis StephanieShishis changed the title Modified script from original instructions Shell Assignment #1 (Part 1 & 2) Mar 3, 2026
@StephanieShishis StephanieShishis changed the title Shell Assignment #1 (Part 1 & 2) U of T DSI | Shell - Assignment #1 (Part 1 & 2) Mar 3, 2026
@khsergvl

khsergvl commented Mar 6, 2026

Copy link
Copy Markdown

Great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants